Skip to content

fix: append L suffix to timestamp filter for Exact API compatibility#297

Merged
tobiascadee merged 1 commit intomainfrom
fix/timestamp-filter-long-suffix
Mar 20, 2026
Merged

fix: append L suffix to timestamp filter for Exact API compatibility#297
tobiascadee merged 1 commit intomainfrom
fix/timestamp-filter-long-suffix

Conversation

@tobiascadee
Copy link
Copy Markdown
Contributor

Summary

  • Appends L suffix to the timestamp value in the OData \$filter parameter (e.g. Timestamp gt 12345L)
  • The Exact API requires this suffix to treat the value as a long integer type in OData filter expressions

Test plan

  • Run a sync stream and verify the timestamp filter is correctly formatted with the L suffix
  • Confirm incremental replication still works as expected

🤖 Generated with Claude Code

The Exact API requires timestamp values to be suffixed with 'L' to
denote a long integer type in OData filter expressions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tobiascadee tobiascadee requested a review from a team as a code owner March 20, 2026 11:52
@tobiascadee tobiascadee merged commit 3ab4db2 into main Mar 20, 2026
4 checks passed
@tobiascadee tobiascadee deleted the fix/timestamp-filter-long-suffix branch March 20, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant